Skip to content

Fix @SecondTuesdayOfMonth and change logic for pulling rollups#11

Merged
jonasatgit merged 1 commit into
jonasatgit:masterfrom
Madj42:patch-2
Apr 13, 2022
Merged

Fix @SecondTuesdayOfMonth and change logic for pulling rollups#11
jonasatgit merged 1 commit into
jonasatgit:masterfrom
Madj42:patch-2

Conversation

@Madj42

@Madj42 Madj42 commented Mar 2, 2022

Copy link
Copy Markdown

Fixed @SecondTuesdayOfMonth and change logic for pulling the current and last rollups.

Fixed @SecondTuesdayOfMonth and change logic for pulling the current and last rollups.
@Madj42

Madj42 commented Mar 2, 2022

Copy link
Copy Markdown
Author

This is the corrected pull request.

@Madj42

Madj42 commented Mar 3, 2022

Copy link
Copy Markdown
Author

I'm not really sure why I was having issues with the second Tuesday query but today it's working with the original version. So I guess you can take that with a grain of salt. I still however think the other part of the changes are beneficial as it makes the compliance percentages more accurate based on the time of month and they do not stop working on the first of the month before patch Tuesday.

@jonasatgit

Copy link
Copy Markdown
Owner

Hi @Madj42 thank you for your contribution. I haven't had time to look into it yet but will do asap.
Best regards
Jonas

@Madj42

Madj42 commented Apr 5, 2022 via email

Copy link
Copy Markdown
Author

@jonasatgit jonasatgit left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution. Highly appreciated.
I will update the whole solution in the next couple of days to introduce some more changes and will update the blog article to reflect that.
Best regards
Jonas

@jonasatgit jonasatgit merged commit be470aa into jonasatgit:master Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants